rename relations for SysML compatibility by RSingh1511 · Pull Request #446 · eclipse-score/docs-as-code

RSingh1511 · 2026-03-23T07:31:57Z

📌 Description

🚨 Impact Analysis

This change does not violate any tool requirements and is covered by existing tool requirements
This change does not violate any design decisions
Otherwise I have created a ticket for new tool qualification

✅ Checklist

Added/updated documentation for new or changed features
Added/updated tests to cover the changes
Followed project coding standards and guidelines

github-actions · 2026-03-23T07:32:39Z

License Check Results

🚀 The license check job ran with the Bazel command:

bazel run --lockfile_mode=error //src:license-check

Status: ⚠️ Needs Review

Click to expand output

[License Check Output]
Extracting Bazel installation...
Starting local Bazel server (8.6.0) and connecting to it...
INFO: Invocation ID: 8fe205c3-85e5-4a36-b0b7-a770c418853e
Computing main repo mapping: 
Computing main repo mapping: 
Loading: 
Loading: 0 packages loaded
Loading: 0 packages loaded
Loading: 0 packages loaded
    currently loading: src
Loading: 0 packages loaded
    currently loading: src
WARNING: Target pattern parsing failed.
ERROR: Skipping '//src:license-check': no such target '//src:license-check': target 'license-check' not declared in package 'src' defined by /home/runner/work/docs-as-code/docs-as-code/src/BUILD
ERROR: no such target '//src:license-check': target 'license-check' not declared in package 'src' defined by /home/runner/work/docs-as-code/docs-as-code/src/BUILD
INFO: Elapsed time: 7.429s
INFO: 0 processes.
ERROR: Build did NOT complete successfully
ERROR: Build failed. Not running target

masc2023

Not sure why you want to change it, but before you can change something, you need to apply change request, as this is defined here
https://eclipse-score.github.io/process_description/main/general_concepts/score_building_blocks_concept.html#building-blocks-meta-model

MaximilianSoerenPollak · 2026-03-23T08:44:54Z

We can't just change something big like that.

A lot of the documentations are already build with these namings in mind, and now changing them without any reasoning or agreeing inside S-CORE that this needs to happen does not seem smart nor ideal to me.

As @masc2023 said above, if this truly is a change needed, please make sure you discuss it in the correct meetings inside S-CORE and make a change request.

As it's still on draft I won't close this. But as is currently this will not be considered.

aschemmel-tech · 2026-03-23T12:20:25Z

This change is related to eclipse-score/process_description#418 - which exists and is approved by the process_community since some time, but never got the capacity to do.

MaximilianSoerenPollak · 2026-03-26T08:31:08Z

This change is related to eclipse-score/process_description#418 - which exists and is approved by the process_community since some time, but never got the capacity to do.

Then alright, seems reasonable.

I will review it once it is not in draft anymore.

MaximilianSoerenPollak · 2026-04-08T08:22:54Z

@RSingh1511 this PR has some merge conflicts.
Could you rebase and solve them? Then we can have a look and potentially merge this for next release.

RSingh1511 · 2026-04-08T09:02:06Z

@RSingh1511 this PR has some merge conflicts. Could you rebase and solve them? Then we can have a look and potentially merge this for next release.

Done.

MaximilianSoerenPollak · 2026-04-08T14:28:21Z

There is still some conflicts in requirements.rst as well as the issue with the process that Roland mentioned.

masc2023 · 2026-04-08T14:33:29Z

This change is related to eclipse-score/process_description#418 - which exists and is approved by the process_community since some time, but never got the capacity to do.

Then alright, seems reasonable.

I will review it once it is not in draft anymore.

Again some concerns, the issues says compatibility to SysML, can we already target SyML 2.0, but would that not mean rather using "refine" as derived_from.

github-actions · 2026-04-10T13:33:02Z

The created documentation from the pull request is available at: docu-html

aschemmel-tech

Please check my comment above and inline comments for the necessary changes. Not implemented currently is the change belongs_to -> satisfied_by

MaximilianSoerenPollak · 2026-06-08T13:28:32Z

If you are changing requirements you will have to increase the version number of that requirements.
Do not delete the version.

aschemmel-tech

Looks good, two minor comments inline

…quirements

aschemmel-tech

my comments resolved

MaximilianSoerenPollak

Some questions, and please do not change bazel_deps without reason.

MaximilianSoerenPollak · 2026-06-10T10:18:34Z

+bazel_dep(name = "rules_python", version = "2.0.2")
+
+# sphinxdocs was extracted into a separate BCR module starting with rules_python 2.0.0
+bazel_dep(name = "sphinxdocs", version = "2.0.2")


Please do not update bazel dependencies if not absolutely needed for your PR.

MaximilianSoerenPollak · 2026-06-10T10:18:59Z

 # (?:, version = ["'][^"']+["'])?                 — optional version argument, any value
 # \)                                              — closing paren
-_BAZEL_DEP_PATTERN = r"""^bazel_dep\(name = ["']score_docs_as_code["'](?:, version = ["'][^"']+["'])?\)"""
+_BAZEL_DEP_PATTERN = r"""^bazel_dep\(name = ["']score_docs_as_code["'][^)]*\)"""


Why is this suddenly changed?

MaximilianSoerenPollak · 2026-06-10T10:43:09Z

As a follow up please make a PR that 'hard' deprecates the satisfies that is 'allowed' in this PR.

MaximilianSoerenPollak · 2026-06-10T10:52:47Z

+  satisfies:
+    incoming: satisfied by
+    outgoing: satisfies


Should be deprecated in a new PR after this one

Agreed — tracked in follow-up issue #594

AlexanderLanin · 2026-06-10T10:54:12Z

@RSingh1511 After this is merged, can you provide a PR which removes all the deprecated options?

RSingh1511 · 2026-06-10T15:13:29Z

@RSingh1511 After this is merged, can you provide a PR which removes all the deprecated options?

Done — added a TODO

github-project-automation Bot added this to INF - Infrastructure Community Mar 23, 2026

github-project-automation Bot moved this to Backlog in INF - Infrastructure Community Mar 23, 2026

masc2023 requested changes Mar 23, 2026

View reviewed changes

masc2023 mentioned this pull request Mar 23, 2026

rename relations for SysML compatibility eclipse-score/process_description#618

Merged

RSingh1511 requested a review from aschemmel-tech March 24, 2026 04:40

RSingh1511 force-pushed the rs/rename-relations-codebeamer branch 3 times, most recently from dd364de to 9763c60 Compare March 24, 2026 10:34

RSingh1511 requested a review from rmaddikery March 24, 2026 12:25

rmaddikery marked this pull request as ready for review April 7, 2026 08:17

rmaddikery requested review from AlexanderLanin, MaximilianSoerenPollak, dcalavrezo-qorix and nradakovic as code owners April 7, 2026 08:17

RSingh1511 force-pushed the rs/rename-relations-codebeamer branch from 9763c60 to 5896de3 Compare April 7, 2026 11:42

RSingh1511 force-pushed the rs/rename-relations-codebeamer branch from 5896de3 to 1352ab7 Compare April 8, 2026 09:00

RSingh1511 requested a review from masc2023 April 8, 2026 09:00

RolandJentschETAS requested changes Apr 8, 2026

View reviewed changes

Comment thread src/extensions/score_metamodel/metamodel.yaml

RSingh1511 requested a review from RolandJentschETAS April 10, 2026 13:16

RSingh1511 force-pushed the rs/rename-relations-codebeamer branch from 9ce6d30 to beb6806 Compare April 10, 2026 13:27

RSingh1511 requested a review from mobileinfo April 14, 2026 11:13

aschemmel-tech requested changes Jun 8, 2026

View reviewed changes

Comment thread docs/internals/requirements/requirements.rst

Comment thread docs/internals/requirements/process_overview.rst Outdated

RSingh1511 requested a review from aschemmel-tech June 9, 2026 03:24

RSingh1511 force-pushed the rs/rename-relations-codebeamer branch 2 times, most recently from eb3dd0d to 790e0a2 Compare June 9, 2026 04:15

RolandJentschETAS previously approved these changes Jun 9, 2026

View reviewed changes

RSingh1511 force-pushed the rs/rename-relations-codebeamer branch from 790e0a2 to 3b22afa Compare June 9, 2026 11:15

aschemmel-tech requested changes Jun 9, 2026

View reviewed changes

Comment thread src/extensions/score_metamodel/metamodel.yaml Outdated

Comment thread src/extensions/score_metamodel/metamodel.yaml Outdated

RSingh1511 dismissed RolandJentschETAS’s stale review via 1c570a7 June 9, 2026 12:32

RSingh1511 requested review from RolandJentschETAS and aschemmel-tech June 9, 2026 12:32

RSingh1511 force-pushed the rs/rename-relations-codebeamer branch from 1c570a7 to 3b4398b Compare June 9, 2026 12:36

RSingh1511 added 4 commits June 9, 2026 14:38

rename relations for Codebeamer compatibility

2555455

update test RST files to use new relation names

815a36b

Revert fulfils to satisfies on arch elements, add satisfied_by for re…

a5ff38b

…quirements

restrict renames to feat_req/comp_req, restore version fields

d934910

RSingh1511 force-pushed the rs/rename-relations-codebeamer branch from 3b4398b to be70da8 Compare June 9, 2026 12:43

fix sphinxdocs load for rules_python 2.0.2

954968a

RSingh1511 force-pushed the rs/rename-relations-codebeamer branch from be70da8 to 954968a Compare June 9, 2026 13:25

RolandJentschETAS approved these changes Jun 10, 2026

View reviewed changes

aschemmel-tech approved these changes Jun 10, 2026

View reviewed changes

masc2023 approved these changes Jun 10, 2026

View reviewed changes

MaximilianSoerenPollak requested changes Jun 10, 2026

View reviewed changes

MaximilianSoerenPollak reviewed Jun 10, 2026

View reviewed changes

Comment thread src/extensions/score_metamodel/metamodel.yaml

MaximilianSoerenPollak reviewed Jun 10, 2026

View reviewed changes

RSingh1511 requested a review from MaximilianSoerenPollak June 10, 2026 15:16

RSingh1511 mentioned this pull request Jun 10, 2026

Hard-deprecate :satisfies: for feat_req/comp_req after migration to :derived_from #594

Open

Conversation

RSingh1511 commented Mar 23, 2026

📌 Description

🚨 Impact Analysis

✅ Checklist

Uh oh!

github-actions Bot commented Mar 23, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

License Check Results

Uh oh!

masc2023 left a comment

Choose a reason for hiding this comment

Uh oh!

MaximilianSoerenPollak commented Mar 23, 2026

Uh oh!

aschemmel-tech commented Mar 23, 2026

Uh oh!

MaximilianSoerenPollak commented Mar 26, 2026

Uh oh!

MaximilianSoerenPollak commented Apr 8, 2026

Uh oh!

RSingh1511 commented Apr 8, 2026

Uh oh!

Uh oh!

MaximilianSoerenPollak commented Apr 8, 2026

Uh oh!

masc2023 commented Apr 8, 2026

Uh oh!

github-actions Bot commented Apr 10, 2026

Uh oh!

aschemmel-tech left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

MaximilianSoerenPollak commented Jun 8, 2026

Uh oh!

aschemmel-tech left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

aschemmel-tech left a comment

Choose a reason for hiding this comment

Uh oh!

MaximilianSoerenPollak left a comment

Choose a reason for hiding this comment

Uh oh!

MaximilianSoerenPollak Jun 10, 2026

Choose a reason for hiding this comment

Uh oh!

MaximilianSoerenPollak Jun 10, 2026

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

MaximilianSoerenPollak commented Jun 10, 2026

Uh oh!

MaximilianSoerenPollak Jun 10, 2026

Choose a reason for hiding this comment

Uh oh!

RSingh1511 Jun 10, 2026

Choose a reason for hiding this comment

Uh oh!

AlexanderLanin commented Jun 10, 2026

Uh oh!

RSingh1511 commented Jun 10, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

6 participants

github-actions Bot commented Mar 23, 2026 •

edited

Loading